Prior to this PR, if you pass in a database connection to querychat_app() it will close your connection for you. You only would want that if querychat created the table connection internally, i.e. if you passed in a data.frame.

Before

pkgload::load_all()

con <- DBI::dbConnect(duckdb::duckdb())
duckdb::dbWriteTable(con, "mtcars", mtcars)

querychat_app(con, "mtcars")
#> Closing data source...

DBI::dbGetQuery(con, "SELECT * FROM mtcars LIMIT 5;")
#> Error in `dbSendQuery()`:
#> ! Invalid connection
#> ℹ Context: rapi_prepare

After

The database connection remains intact and the dbGetQuery() call completes as expected.